-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[host] Copy hostfxr and hostpolicy for Mono also, if !mobile #73633
[host] Copy hostfxr and hostpolicy for Mono also, if !mobile #73633
Conversation
Fixes the mono "HelloWorld" sample (src/mono/samples/HelloWorld) on desktop platforms
Tagging subscribers to this area: @directhex Issue DetailsFixes the mono "HelloWorld" sample (src/mono/samples/HelloWorld) on desktop platforms
|
<!-- copy the host files to make the desktop samples work --> | ||
<RuntimeFiles Include="@(HostFxrFile)" Condition="Exists('@(HostFxrFile)') and '$(TargetsMobile)' != 'true'"/> | ||
<RuntimeFiles Include="@(HostPolicyFile)" Condition="Exists('@(HostPolicyFile)') and '$(TargetsMobile)' != 'true'" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These files are already copied above in L48-56. @agocke can you please take a look?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that's for the test host.
The change in this PR is for the "runtime pack" that is used by eng/targetingpacks.targets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's wait for @agocke's review as he just recently made the same change just a bit differently.
This comment was marked as off-topic.
This comment was marked as off-topic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just saw that the same already exists for CoreClr in the OverrideRuntimeMono
target so it should be OK.
Fixes the mono "HelloWorld" sample (src/mono/samples/HelloWorld) on desktop platforms